Skip to content

Fix: added information about query parameters validation #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 4, 2019

Conversation

IgorSasovets
Copy link
Contributor

No description provided.

@PauloASilva
Copy link
Collaborator

Hi @IgorSasovets,
First of all thanks for your contribution!

By mistake, I suggested adding your changes to the
2019/en/src/0xa3-improper-data-filtering.md file: I should have written
2019/en/src/0xa4-lack-of-resources-and-rate-limiting.md. Can you please move
your changes to the appropriate file?

I also have two suggestions to improve your attack scenario:

  • Remove reference to "MEAN stack": we want to keep this document agnostic and
    this same scenario is also valid for other stacks.
  • "it causes performance issues": add one or more examples of what can cause
    such performance issues e.g. database timeout, memory exhaust, ...

Cheers,
Paulo A. Silva

@IgorSasovets
Copy link
Contributor Author

Hi, @PauloASilva !

I added suggested changes. Please take a look at them.

Best regards,
Ihor

@PauloASilva PauloASilva merged commit 55eb86a into OWASP:develop Jul 4, 2019
PauloASilva pushed a commit that referenced this pull request Jan 22, 2021
Update 0x04-release-notes.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants